Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix/cli argparse #14053

Merged
merged 6 commits into from
Jun 9, 2023
Merged

Fix/cli argparse #14053

merged 6 commits into from
Jun 9, 2023

Conversation

memsharded
Copy link
Member

@memsharded memsharded commented Jun 8, 2023

Changelog: Fix: Allow internal Cli object to be called more than once.
Docs: Omit

The main change is a refactor, not creating the ArgumentParser when the commands are imported, but when the commands are executed, because otherwise it is not possible to call a Cli.run() again.

Close #14044

@memsharded memsharded added this to the 2.0.7 milestone Jun 8, 2023
@memsharded memsharded marked this pull request as ready for review June 8, 2023 22:45
@czoido czoido merged commit 2202b35 into conan-io:release/2.0 Jun 9, 2023
@memsharded memsharded deleted the fix/cli_argparse branch June 9, 2023 07:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[bug] Running multiple commands via the CLI v2 API raises ArgumentError
2 participants